Modernize Humminbird internals (#1322)

* Modernize Humminbirds internals

---------

Co-authored-by: Robert Lipe <robertlipe@gpsbabel.org>

use implicit conversions for lat,lon degs<->rads in grtcirc interface (#1321)

* use implict conversions for lat,lon degs<->rads.

* use Position* return values.

* convert grtcirc i/f to PositionDeg, PositionRad.

* fix mac compile, denote converting ctor.

* inline PositionDeg/Rad converting ctors.

* respect PosotionX encapsulation.

* differentiate member names for PositionDeg, PositiionRad.  This can catch unintended usages where the type is not what was expected.

* clean up unneccesary usage of Waypoint::position.

* delete obsolete declaration.

* spell

* update garmin include

* consistenly unpack Positions to consts in grtcirc.

* provide default initializers for PositionX.

fix bend filter usage of linepart. (#1320)

* fix bend filter usage of linepart.

* fix new bend filter test.

* make new reference files valid kml.

* debug macos fails

* workaround rounding errors in problematic kml lookat range.

cleanup great circle. (#1318)

* cleanup great circle.

* fix whitespace

correct simplify filter with maximum allowable error option and the metric being overall route length or relative error (#1316)

* add test for simplify w/ relative+error opts.

* fix simplify filter error+length operation.

* fix simplify relative+error

* remove debug statement

* simplify error unit tests and documentation.

* format parse, update includes.

* prune redundant test case

have jeeps use std::endian (#1315)

* have jeeps use std::endian

* detect unsupported endian platforms.

move floor to C++20 (#1206)

* move floor to c++20

drop Visual Studio 2017.
use default member intializers for bit-fields.

* use more bit-field default initializers

* try Xcod 14.3.1 on macOS 13.

* echo clang version

which can be useful with homebrew fails_with

* use std::numbers

* finish eradication of M_PI.

* fix some whitespace

use constexpr for constants, match parms in declaration and definition. (#1314)

* use constexpr for constants, match parms in declaration and definition.

* increase precision of GPS_PI.

Use Helmert transform for conversions between OSGB36 and WSG84 (#1311)

* use Helmert transform for OSBG36 <-> WGS84.

* adjust unicsv_grids reference for Helmert xform.

* update copyright.

* use constexpr to select helmert inversion method.

* tweak helmert.

spell Datum (#1313)

unify access to ellipsoid parameters (#1312)

* define constants for often used spherioid parms.

* add function to calculate semi-minor axis.

* use semi major axis function more.

* have semi_minor_axis consume an ellipse

* compute semi minor axis in GPS_Ellipse class.

add constants for other ellipses that are used directly.

correct WGS84 semi minor axis value. (#1310)

It appears the GSR80 value was used instead of the WGS84 value, which
is a very small error.
However, there was a typo in the value in GPS_Math_XYZ_To_WGS84LatLonH
that resulted in an order of magnitude error!
This could lead to GPS_Math_XYZ_To_WGS84LatLonH never converging.
This may be why skytraq created their own function to convert ECEF
to WGS84.

cleanup datetime.h tidy and g++ warnings. (#1309)

Wsign-conversion
readability-implicit-bool-conversion
readability-else-after-return
cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers

enhance validate filter to report on track segments. (#1308)

use a signed type for waypt_counts and route counts. (#1307)

* use a signed type for waypt_counts and route counts.

Although larger in Qt6, the underlying type is signed.

* fix -Wformat-signedness warnings introduced by this PR.

* fix new -Wsign-conversion warnings.

teach lowranceusr about track segments (#1306)

* teach lowranceusr about track segments.

lowranceusr versions 2 & 3 have a continuous flag used to
indicate dicontinuities in a trail.
The lowranceusr version 2 & 3 reader now flags trail discontinuities
as new track segements.
The lowranceusr version 2 & 3 writer now flags new track segments as
discontinuities.

Also fix two subtle bugs:
1) when reading a version 2 or 3 trail with the break option we would
fail to break the trail if the discontinuity was at the start of a
section other than the first section.
2) when writing a version 2 or 3 trail with the merge option if the
number of points exceeded MAX_TRAIL_POINTS we would write a header
that indicated MAX_TRAIL_POINTS but actually write MAX_TRAIL_POINTS + 1.

* add lowranceusr test for track segments.

Update Russian translation (#1304)

* Update Russian translation

* Update Russian translation

correct garmin real time altitude, add geoid height. (#1303)

correct computation of timestamp with garmin real time positioning. (#1302)

* correct computation of timestamp with garmin real time positioning.

* tweak garmin pvt conversion

fix gpsapp copy_char_array that could leave part of a packet uninitialized. (#1301)

* fix gpsapp copy_char_array that could leave part of a packet uninitialized.

* tidy up a bit.

* delete empty statement

fix memory leaks writing garmin routes. (#1299)

enhance garmin route write for devices using 16 member icon set. (#1298)

enhance garmin route writes for devices using D201_Rte_Hdr_Type. (#1297)

which use GPS_PWay->rte_cmnt for the route name.

don't send links with GPS_A200_Send. (#1295)

fix faulty assertion in garmin. (#1290)

the '-' character is legal for some devices, but was
excluded in the assertions as it can be a metacharacter.

remove qtci installer code (#1289)

* remove qtci which hasn't been used for a long time.

* remove another obsolete qtci file.

* remove another qtci reference.

* delete obsolete travis build code.

* rename osx install script

* catch ci up with osx travis install script renames.

* remove obsolete debug code from qtio installer.

exclude qt .ts files from codacy analysis. (#1288)

they confuse eslint which thinks they are TypeScript.

Fix unicsv reader delimiter detection (#1287)

* Fix unicsv reader delimiter detection.

Any delimiter sequence that is enclosed has never been considered
a delimiter when splitting a line into fields.  However, previously
an enclosed potential delimiter could be detected when scanning the
first line, and then erroneously used as the delimiter.

* tweak auto

* add unicsv delimiter detection test

delete obsolete jeeps functions that were never used (#1285)

since introduction in 2002.

update macos release to Qt 6.5.3 (#1284)

refactor interface to jeeps pvt related commands. (#1283)

fix xcsv RECORD_DELIMITER CR. (#1282)

add Qt's webengine translations to package. (#1277)

tidy readability-convert-member-functions-to-static (#1276)

clang-tidy modernize-loop-convert (#1275)

clang-tidy readability-isolate-declaration (#1274)

Move floor to Qt 6.2 (#1272)

* move floor from Qt 5.15 to 6.2.

* fix some CI failures

* try harder to fix coverage build

* try to fix coverage again

* try agin

* move up msvc tools minimum for Qt6

* clean up clazy warnings with Qt6.

* drop redundnant CI job

* revert translation related unintentionl changes

* simplify find qt packages

* assume Qt6 in textstream

* move CI tidy to noble

* assume Qt6 in textstream.h

* remove obsolete makesetup.bat

* remove Qt5 support from coretool

* strip Qt5 from noble image

* update default qt versions.

* update minimum cmake version to match Qt 6.2.4 QT_SUPPORTED_MIN_CMAKE_VERSION_FOR_USING_QT

* assume QHash result is type size_t

* remove unnecessary qOverloads (in Qt6).

add Ubuntu noble to CI (#1273)

* add ubuntu noble docker support

* update qt6 packages for noble.

noble has some new qt6 packages that alleviate the need to
directly install some libraries.
also, the dependencies of some qt6 packages have been improved
allowing us to specify what we use more directly.
this results in the same set of packges being installed.

* remove dependent qt5 package

* add noble to CI

add fedora 40 to CI (#1270)

support builds with shapelib (as documented). (#1268)

update libusb to 1.0.27 (#1259)

work around homebrew python linking issues (#1267)

* debug python brew issues

* dbg2

* dbg3

* dbg4

* insulate workflow from homebrew location.

Add test coverage for ozi color setting options. (#1266)

* Unit test ozi color commandline options.

Improves test coverage.

Co-authored-by: Robert Lipe

update macos github workflow (#1265)

* update github macos runners.

macos-11 runner is now deprecated.

* update xcode versions for github

* update XML_CATALOG_FILES for macos-14 runner

* work around macos-14 runner brew locations

* find homebrew gsed on apple silicon

* generalize brew location detection in macos workflow

* update python version for arm availability

* push xcode versions to latest minor levels.

clang tidy 18 fixes readability-redundant-casting (#1262)

update zlib to 1.3.1 (#1258)

* update zlib to 1.3.1

* delete extra zlib files

update shapelib to 1.6.0 (#1257)

Bump actions/cache from 3 to 4 (#1256)

Bumps [actions/cache](https://github.com/actions/cache) from 3 to 4.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](https://github.com/actions/cache/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

update codacy badge (#1251)

update codeql workflow template (#1250)

Bump actions/upload-artifact from 3 to 4 (#1249)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3 to 4.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump github/codeql-action from 2 to 3 (#1248)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2 to 3.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/v2...v3)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

modernize google maps javascript (#1247)

to avoid:
"js: Google Maps JavaScript API has been loaded directly without a callback. This is not supported and can lead to race conditions and suboptimal performance. For supported loading patterns please see https://goo.gle/js-api-loading"

remove CMakeSettings.json (#1246)

which had machine specific windows configuration values.

Bump actions/setup-python from 4 to 5 (#1245)

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](https://github.com/actions/setup-python/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

make xmlgeneric a class (#1244)

* make xmlgeneric a class.

* encapsulate more of xmlgeneric

* use scoped enum

* encapsulate a bit more.

* retire xg_string type

* use scoped enums

* rule of zero applied to xmlgeneric class.

* remove test code

fix errors in gpi writer related to bitmaps. (#1243)

Erroneous warnings "garmin_gpi: Code error in load_bitmap_from_file,
expected output size 1060, actual output -28."

Corrupted colors when input bitmap has 24 bits per pixel.

bring static functions into googltakeout class. (#1240)

and other tidy modernizations.

straggling nits (#1236)

* actually quit using gpsfmt.h

* use auto

* empty statements

* join declaration and assignment

* fix tab

* make member funct static

* delete renamed prototypes

* const pointers

* more const pointers

* empty statements

misc cleanups (#1235)

simplify usage of homebrew on CI macos-11. (#1234)

homebrew installs are taking an extreme amount of time on macos-11,
with the following message:
  Warning: You are using macOS 11.
  We (and Apple) do not provide support for this old version.
  It is expected behaviour that some formulae will fail to build in this old version.
  It is expected behaviour that Homebrew will be buggy and slow.
To alleviate this pain skip the installation of jing and all it's
dependencies on macos-11.

retire LegacyFormat class. (#1233)

convert mtk formats to Format class. (#1226)

convert ozi to Format class. (#1232)

* convert ozi to Format class.

There is a subtle difference in the writer, the index is now
reset for every invocation.  This only matters if the writer is
used more than once. I believe this was the intent.

* fool with QIODevice and it's inheritence of QIODeviceBase.

convert garmin_txt to Format class (#1231)

* convert garmin_txt to Format class

* fix includes for qt5

convert garmin_xt to Format class (#1230)

convert tpg to dynamic Format class. (#1229)

convert gtm to Format class. (#1228)

convert v900 to dynamic Format class. (#1227)

convert tpo formats to Format class (#1225)

* convert tpo to Format class.

* capture comments

fix leak in jeeps serial comm on posix. (#1223)

see #1221

convert garmin to Format class (#1222)

* convert garmin to format class.

* add new file garmin.h

* make some member fns static.

* init member var

astyle jeeps included files (#1219)

gpsmath isolation (#1218)

gpsmath/gpsdatum/gpsproj don't have dependencies
on the rest of jeeps.

cleanup typedefs, macros, const tables.

use c++11 fixed width integer types in jeeps. (#1217)

* use c++11 fixed width integer types in jeeps.

thanks resharper!

* fix fixed width integers resharper missed.

in non windows code.

use uniform header guards in jeeps. (#1216)

and
delete an obsolete prototype.
rename jgpsutil.cc now that it doesn't conflict with a format.

jeeps function pointers, GPS_PPacket const correctness, GPS_Serial_[OSP]Packet -> GPS_Serial_Packet (#1215)

* use using for jeeps function prototypes.

* const correctness wrt GPS_PPacket.

The important bit here is in gpssend.cc, Build_Serial_Packet used to rececive a 1KB packet by copy, now it is by const reference.

* delete accidentally committed file.

* rename GPS_PPacket -> GPS_Packet.

original jeeps used GPS_SXXX, GPS_OSXXX, GPS_PXXX for the structure
tag, a typedef of the structure, and a pointer to a typedef of the
structure.  In the case of GPS_PPacket this convention was no longer
followed.

* change GPS_Packet back to a structure.

* try harder to change GPS_Packet back to a struct.

* clean up GPS_Serial_[SOP]Packet.

* catch some missed renames up.

clion on linux missed some windows only renames.

clang tidy fixes (#1214)

* fix tidy readability-named-parameter

* fix readability-named-parameter fix

* fix some readability-inconsistent-declaration-parameter-name tidy warnings.

* update tidy script to

ignore strptime
ignore third party headers
clean build directory
generate summary

* update git ignores

misc cleanups (#1213)

initialize garmin_fs class member.
catch gui up with Qt 5.15 floor.
rename tidy build directory.

make qstarz format dynamic (#1212)

add fedora 39 to ci (#1211)

* add fedora 39 to ci

* retire qAsConst for std::as_const

convert vcf to dynamic Format. (#1210)

refactor garmin_fs class (#1203)

* use container for garmin_fs ilinks.

and leave memory management up to the copy on write container.

* make gmsd const for read only usage.

this prevents the possiblity of detachment of the ilinks list.

* incorporate some variables and functions into garmin_fs_t

* unify garmin category conversions.

* fiddle with test time zone sensitivity.

* add utc option for garmin_txt reader.

* correct sign of adjustment

* update garmin_txt includes.

convert garmin_fs functions from char* -> QString (#1202)

* convert garmin_fs functions from char* -> QString.

* update comment

* make tag_type a class enum

* update garmin_fs includes

consolidate handling of extensions in gpx writer. (#1200)

This is a user visible change.

Previously if gARmIN sPECIAL dATA existed and both garminextensions
and humminbirdextensions were false then the gpx writer would
write a subset if it (and not passthrough any gpx -> gpx data.)
If gARmIN sPECIAL dATA existed and either extension option was true,
then a different subset would be written (and any gpx -> gpx data would
not be passed through.)  Now the garminextensions option must
used to have any gARmIN sPECIAL dATA data written.

Previously, when passing through extension data from the gpx reader to
the gpx writer, some elements of the garmin and humminbird extensions
were not included. Now, all foreign elements are included. As before,
passthrough is only used when both extension options are false.

Develop igc options (#1180)

* Individual options for IGC extensions

* Remove unused var, address compiler complaint

* Minor cleanup

* Remove ALL and EVERY options, make defaults more obvious

* Simplify debug code, refactor maps, add test cases (<--tseven4 patch)

* Make QT5 happy

* capture serialization usage.txt file verbatim.

---------

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

move Qt floor from 5.12 to 5.15 (#1199)

* move Qt floor to 5.15.

* fix focal, coverage, windows 32 bit builds.

* try harder to fix coverage build

* fix codeql build

* kill fedora 32 CI build which uses Qt 5.14.

support optional start elements in xmlstreamwriter. (#1193)

this adds methods to xmlstreamwriter to allow conditional writing
of a start element.  The conditional element will only be written to
the stream if it has children, which is deteremined when the condtional
element is ended.  This simplifies writing of such elements as the
writer does not need to know if it will subsequently write children or
not.

remove gpx reader support for probelmatic extensions. (#1198)

1. groundspeak is generally used with gpx 1.0.  Our attempt to
support it with gpx 1.1 required schema violations.

2. opencaching.de no longer uses the geocache element.  Our
implementation required schema violations.

fix bugs with gpx passthrough. (#1197)

with gpx 1.0, any extension data that was a child of rte or trk
was lost.  I beleive this used to work but has been broken for quite
a while.

with gpx 1.0 extension data that was a child of gpx could be
erroneously rewritten as part of the final wpt/rtept/trkpt.

with gpx 1.1 extension data that was a chile of trkseg could be
erroneously rewritten as part of the final wpt/rtept/trkpt.

I note we currently drop any extension data that is a child of gpx,
,a child of trkseg, or a child of metadata.

Update (now archived) Oozy Rat from Geocaching.com in 2023 (#1196)

* Update (now archived) Oozy Rat from Geocaching.com in 2023

* Replace GCGCA* (Ooozy Rat) with 2023 version from Groundspeak.

* Regenerate all reference files.

* Nail TZ in geocache logs in text, html to GMT.

This has been a problem that's plagued geocaching apps forever

---------

Co-authored-by: Robert Lipe <robertlipe@gmail.com>

enhance gpx test to validate gpx writer. (#1191)

* enhance gpx test to validate gpx writer.

this reveals our writer producing a schema violation, an exception
is made to avoid testing with this input.  The exception should
be removed when we fix the gpx writer.

* fix write order of gpx elements we just pass through.

* collapse writing of waypoint elements common to wpts, rtepts, trkpts.

* fiddle

* more fiddling

rc1 for release 1.9.0 (#1186)

archive docs in CI (#1187)

* create document generation workflow

* fix webdocdir bug

* update checkout

* update document for cmake variable renames.

bump qt build from 6.5.2 to 6.5.3 (#1185)

Remove final remnants of 'exported' (#1146)

* Remove final remnants of 'exported'

* incorporate review comments in duplicate filter,

and remove deleted exported keyword from unicsv doc.

---------

Co-authored-by: Robert Lipe <robertlipe@gmail.com>

tidy fixes (#1179)

* tidy readability-convert-member-functions-to-static

* tidy readability-make-member-function-const

* kill trailing space

fix various "Redundancies in code"  (#1178)

* fix "Redundant elaborted type specifier"

* fix "Redundant 'inline' specifier"

* fix "redundant 'const' specifier on a constexpr variable"

* remove "Redundant qualifier"

* add /out/, /.vs/ to .gitignore, and sort.

fix SchemaArrayField/SchemaArrayData mismatch (#1177)

* fix SchemaArrayField/SchemaArrayData mismatch

related to IGC additions from #1060 and #1054.

* retire global trait collection

* use class member to store track traits instead of fs data.

* fiddle with kml accumlate hash

* compute number of wp_fields

* correct bug concerning track specific traits.

* simplify multi-track array decisions.

* simplify kml multi-track schema decisions.

Integrate OAT and SIU with Waypoints (#1060)

* Integrate OAT and SIU with Waypoints

* Type corrections, convert set_value() overload to nullptr* check

* #1062 Fix core dump when merging tracks with disparate IGC extensions

* Reduce `value` variable scope when writing SimpleArrays

* Debug oopsie bugfix, fix core dump when merging IGC files with no extensions

* Collapse redundant if statement

* Fix brain fart

* Eliminate igc_fsdata::has_value()

* Fix subtle satellite count bug

* Fix small sat count bug, and eliminate some ifdefs

* Make some IGC exts override default WP members

* Include IGC line-by-line debug info

* Remove accidentally included file

* delete obsolete macros

---------

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>